Skip to content

Java02. ДЗ 01, Кравченко Юрий, подгруппа 2#1

Open
XJIE6 wants to merge 6 commits intomasterfrom
hw1
Open

Java02. ДЗ 01, Кравченко Юрий, подгруппа 2#1
XJIE6 wants to merge 6 commits intomasterfrom
hw1

Conversation

@XJIE6
Copy link
Copy Markdown
Owner

@XJIE6 XJIE6 commented Feb 13, 2016

No description provided.

@sproshev
Copy link
Copy Markdown

Не надо коммитить:

  1. папку .idea и *.iml файлы
  2. папку target

Правило такое: если что-то можно сгенерировать на основе исходников или каких-то конфигов, то это что-то не надо коммитить )


public class LazyFactory {

abstract static class MyLazy<T> implements Lazy {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

private ... implements Lazy<T>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-0.5

@XJIE6
Copy link
Copy Markdown
Owner Author

XJIE6 commented Feb 14, 2016

@sproshev

  1. Мы просили руководство, чтобы нам провели ликбез по гиту, но так ничего и не добились. Было бы классно, если бы вы рассказали нам про .gitignore
  2. Я исходил из того, что тестов много не бывает. Удобно иметь как простые, так и сложные тесты (которые могут полностью покрывать простые), ведь если упадёт простой, то найти ошибку проще. Тесты с Null проверяют, что внутри не используется сравнения с Null. Тесты с Supplier проверяют, что внутри нет instanceOf Supplier

public class LazyFactory {

abstract private static class MyLazy<T> implements Lazy<T> {
Boolean got = false;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boolean -> boolean

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants